-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[MOB-9899]: Add Auth Checks Before API Calls #463
Conversation
…AUT_main_bug_bash
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I forget that js modules are basically singletons so attributes are global like this 😅 but looks good, just left one callout!
src/embedded/embeddedManager.ts
Outdated
if (typeOfAuth !== null) { | ||
await this.retrieveEmbeddedMessages(packageName, placementIds || []); | ||
callback(); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we throw an error in the case of no auth?
JIRA Ticket(s) if any
Description
Adds
typeOfAuth
verification before API calls. With AUT, it's possible folks try to make these calls before being initialized more frequently. Should be helpful regardless of AUT.Test Steps